Skip to content

[codex] Simplify README demo links - #5

Closed
lishehao wants to merge 1 commit into
mainfrom
codex/simplify-demo-links
Closed

[codex] Simplify README demo links#5
lishehao wants to merge 1 commit into
mainfrom
codex/simplify-demo-links

Conversation

@lishehao

@lishehao lishehao commented Jun 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Simplifies the README demo section so reviewers see one primary YouTube path: a clickable poster plus a single Watch on YouTube CTA.
  • Removes the repeated raw YouTube URL, embedded local video block, separate local MP4 link, and separate poster-frame link.
  • Keeps the local MP4 as a quiet offline fallback.
  • Preserves the no-hardware boundary: this is a real Tkinter UI replay of a deterministic 72-point simulated fixture, not live hardware validation, and production AWG/oscilloscope adapters are not used for the demo.

Validation

  • git diff --check origin/main..HEAD
  • Verified README references https://youtu.be/fYokRzNnm84
  • Verified README no longer contains the local <video> block or repeated fallback/poster links
  • Verified local fallback files exist:
    • docs/images/auto-load-off-test-point-replay-demo.mp4
    • docs/images/auto-load-off-test-point-replay-demo.png

Notes

No existing repo-local GitHub Pages/demo/profile HTML surface was found, so this PR only simplifies the README surface.

@lishehao

lishehao commented Jun 2, 2026

Copy link
Copy Markdown
Owner Author

Closed per user decision: this small README-only demo link simplification was pushed directly to main in commit f180328 instead of merging the PR.

@lishehao lishehao closed this Jun 2, 2026
@lishehao
lishehao deleted the codex/simplify-demo-links branch June 2, 2026 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant